src/Dialog/Content/index.tsx (1)

40-44: 考虑为 dialogRef.current 添加空值检查以保持一致性。

与 onPrepare 中新增的保护措施类似，这里访问 dialogRef.current.inMotion 和 dialogRef.current.enableMotion 时也可能遇到 dialogRef.current 为空的情况。建议添加空值检查以提高代码的健壮性。

可以考虑使用以下方式：

 React.useImperativeHandle(ref, () => ({
   ...panelRef.current,
-  inMotion: dialogRef.current.inMotion,
-  enableMotion: dialogRef.current.enableMotion,
+  inMotion: dialogRef.current?.inMotion,
+  enableMotion: dialogRef.current?.enableMotion,
 }));

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

tests/index.spec.tsx (1)

282-292: 新增用例方向正确，但建议减少对其他用例副作用的依赖

这个测试很好地抓住了“nativeElement 不可用时调用 onAppearPrepare 不应抛错”的核心行为，整体思路没问题。但当前有两点可以稍微收紧一下：

1. 第一个 expect(() => { global.onAppearPrepare?.(); }).not.toThrow(); 在本用例内部没有任何 render，它隐式依赖于前面其它用例已经给 global.onAppearPrepare 赋值；如果这个用例单独执行或用例顺序调整，这一段要么什么都不做（onAppearPrepare 为 undefined），要么行为取决于上一个用例的状态，可读性和稳健性都稍弱。
2. 第二次调用前虽然有一次 render(<Dialog visible />)，但可以考虑在这里也显式构造和本 bug 更接近的场景（例如带 mousePosition 的 Dialog），让测试意图更直接。

可以参考下面两种小改法择一：

• 如果只关心“当前实现下不抛错”，建议去掉对前一个用例状态的依赖：

-    it('should not throw error when nativeElement is not available', () => {
-      expect(() => {
-        global.onAppearPrepare?.();
-      }).not.toThrow();
-
-      render(<Dialog visible />);
-
-      expect(() => {
-        global.onAppearPrepare?.();
-      }).not.toThrow();
-    });
+    it('should not throw error when nativeElement is not available', () => {
+      render(<Dialog visible />);
+
+      expect(() => {
+        global.onAppearPrepare?.();
+      }).not.toThrow();
+    });

• 如果希望覆盖“前一次动画留下的 onAppearPrepare + 当前没有 nativeElement”的组合场景，可以在本用例内先显式渲染一次、再触发关闭或卸载，再做第一次调用，这样不用依赖其它测试的执行顺序，同时场景更自说明。

整体来说，用例已经能保护这次修复，以上只是为了让测试更加自洽、抗变更性更好一些，可按需要取舍。

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro